Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quick fixes #27

Merged
merged 4 commits into from
Dec 11, 2024
Merged

Quick fixes #27

merged 4 commits into from
Dec 11, 2024

Conversation

crimsonknave
Copy link
Collaborator

Updates dependencies to close #26 which fails CI due to some dependency conflicts. Also fix two references to the previous name of the package.

@Copilot Copilot bot review requested due to automatic review settings December 11, 2024 21:58
@crimsonknave crimsonknave requested a review from a team as a code owner December 11, 2024 21:58

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 6 out of 11 changed files in this pull request and generated no suggestions.

Files not reviewed (5)
  • .github/CODEOWNERS: Language not supported
  • requirements/requirements-dev.txt: Language not supported
  • test/test_plugins.py: Evaluated as low risk
  • annotated_logger/init.py: Evaluated as low risk
  • annotated_logger/mocks.py: Evaluated as low risk
Comments skipped due to low confidence (1)

README.md:5

  • The URL for the coverage badge was updated correctly, but the link text still references 'logger-decorator'. It should be updated to 'annotated-logger'.
[![Coverage badge](https://github.com/github/annotated-logger/raw/python-coverage-comment-action-data/badge.svg)](https://github.com/github/annotated-logger/tree/python-coverage-comment-action-data) [![Ruff](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/astral-sh/ruff/main/assets/badge/v2.json)](https://github.com/astral-sh/ruff) [![Checked with pyright](https://microsoft.github.io/pyright/img/pyright_badge.svg)](https://microsoft.github.io/pyright/)

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  annotated_logger
  __init__.py
  mocks.py
Project Total  

This report was generated by python-coverage-comment-action

@crimsonknave crimsonknave merged commit 18e976c into main Dec 11, 2024
21 of 22 checks passed
@crimsonknave crimsonknave deleted the quick-fixes branch December 11, 2024 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants